Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bump fix notifications and mails send after voting #96

Merged
merged 1 commit into from
Aug 26, 2024

Conversation

BarbaraOliveira13
Copy link
Contributor

@BarbaraOliveira13 BarbaraOliveira13 commented Aug 23, 2024

🎩 Description
Bump ptp : Budget Booth, the vote confirmation is neither notified or sent by email.

📌 Related Issues
Notion card

Testing

Dashboard

  1. As an admin, go to a process.
  2. Go to the budget setting (in components, click on the settings icon).
  3. Click on “Vote in one: allows participants to vote in any budget, but only in one.” and “Voting enabled.”
Capture d’écran 2024-07-29 à 13 10 53 Capture d’écran 2024-07-29 à 15 45 36
  1. Update.

Front-end

  1. Go to the process you changed.
  2. Go to budgets.
  3. Start voting (enter the budget booth).
  4. Vote.
  5. Go to your notifications and confirm that you received one about your vote.
  6. Go to http://localhost:3000/letter_opener and confirm that you received an email about your vote.
Capture d’écran 2024-07-31 à 12 05 41

Copy link
Contributor

@Stef-Rousset Stef-Rousset left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it works locally (after launching sidekiq in terminal); I received email and notification !!

@luciegrau luciegrau marked this pull request as ready for review August 26, 2024 13:22
@luciegrau luciegrau merged commit 2f7bb08 into develop Aug 26, 2024
14 checks passed
@luciegrau luciegrau deleted the fix/send_notification_budget_booth branch August 26, 2024 13:22
Quentinchampenois added a commit that referenced this pull request Oct 9, 2024
* fix: Modify default proposal states (#77)

* feat: Addition of Extra User Fields Module (#79)

* feat: Add Extra User Fields

* lint: Rubocop Fixes

* feat: Add Friendly Signup (#78)

* feat: Add friendly signup

* feat: Add configurable options to the config

* fix: Fix some tests and add new ones

* fix: Missing text modification

* chore: Merge Extra user fields

---------

Co-authored-by: quentinchampenois <[email protected]>

* feat: Add module budget booth (#81)

* feat: Add module budget booth

* bump: modules

* ci: Add x86_64 linux support

* test: Explore projects index

* feat: Modules simple proposals & paper ballots (#82)

* fix: Add simple proposals

* lint: Fix rubocop offenses

* fix: Select first photo in proposal form

* test: Add Custom proposals factories

* fix: Downgrade to decidim v0.27.5

* chore: Update db schema.rb

* fix: Add simple proposals migration

* chore: Update db schema.rb

* revert: Gemfile locked to 0.27.5

* ci: Add x86_64 linux support

* test: Fix proposal fields in system specs

* test: Fix simple proposal system specs

* test: Remove broken specs

* test: Remove broken specs

* test: Category id is optional

* bump: Module simple proposal

* feat: Module paper ballots (#83)

* feat: Add module paper ballots

* chore: Update README and schema.rb

* lint: Fix rubocop offenses

* feat: Add module budgets_importer (#84)

* feat:module_budgets_importer

* feat:budgets_importer add/bumb rspec

* delete rspec

---------

Co-authored-by: Quentin Champenois <[email protected]>

* fix: Override budgets importer projects index

* fix: incompat paper ballots and budgets importer

* ci: ignore missing locales

* fix: add french translation key for paper_ballot module (#88)

* fix:budget_importer (#89)

* bump: Bump of simple proposal module (#90)

* fix: Change branch of simple proposal module

* fix: Bump simple proposal module

* fix: Downgrade wicked pdf (#91)

* chore: Update docker compose command

* fix(config): Force Rack attack to false

* Upgrade decidim-custom_proposal_states (#92)

* Update awesome with private fields (#93)

* update awesome with private fields

* use merged branch

* fix: migration FixReferenceForAllResources when table does not exists

* bump fix notifications and mails send after voting (#96)

* Bump: simple_proposal update (#97)

* Bump: simple_proposal update

* test: update system test

* Fix: Bump Decidim Awesome module (#95)

* Fix export dropdown in admin proposals index (#98)

* fix: add missing translation keys

* test: update config file after i18n test

* Bump simple proposal module (#100)

* fix: Add missing Sidekiq queues (#101)

* fix(sidekiq): duplicate sidekiq queue (#102)

The queue `user_report` is duplicated

---------

Co-authored-by: Guillaume MORET <[email protected]>
Co-authored-by: ’Barbara Oliveira <[email protected]>
Co-authored-by: Lucie Grau <[email protected]>
Co-authored-by: Alexandru Emil Lupu <[email protected]>
Co-authored-by: Ivan Vergés <[email protected]>
Co-authored-by: moustachu <[email protected]>
Co-authored-by: stephanierousset <[email protected]>
Quentinchampenois added a commit that referenced this pull request Oct 10, 2024
* fix: Modify default proposal states (#77)

* feat: Addition of Extra User Fields Module (#79)

* feat: Add Extra User Fields

* lint: Rubocop Fixes

* feat: Add Friendly Signup (#78)

* feat: Add friendly signup

* feat: Add configurable options to the config

* fix: Fix some tests and add new ones

* fix: Missing text modification

* chore: Merge Extra user fields

---------

Co-authored-by: quentinchampenois <[email protected]>

* feat: Add module budget booth (#81)

* feat: Add module budget booth

* bump: modules

* ci: Add x86_64 linux support

* test: Explore projects index

* feat: Modules simple proposals & paper ballots (#82)

* fix: Add simple proposals

* lint: Fix rubocop offenses

* fix: Select first photo in proposal form

* test: Add Custom proposals factories

* fix: Downgrade to decidim v0.27.5

* chore: Update db schema.rb

* fix: Add simple proposals migration

* chore: Update db schema.rb

* revert: Gemfile locked to 0.27.5

* ci: Add x86_64 linux support

* test: Fix proposal fields in system specs

* test: Fix simple proposal system specs

* test: Remove broken specs

* test: Remove broken specs

* test: Category id is optional

* bump: Module simple proposal

* feat: Module paper ballots (#83)

* feat: Add module paper ballots

* chore: Update README and schema.rb

* lint: Fix rubocop offenses

* feat: Add module budgets_importer (#84)

* feat:module_budgets_importer

* feat:budgets_importer add/bumb rspec

* delete rspec

---------

Co-authored-by: Quentin Champenois <[email protected]>

* fix: Override budgets importer projects index

* fix: incompat paper ballots and budgets importer

* ci: ignore missing locales

* fix: add french translation key for paper_ballot module (#88)

* fix:budget_importer (#89)

* bump: Bump of simple proposal module (#90)

* fix: Change branch of simple proposal module

* fix: Bump simple proposal module

* fix: Downgrade wicked pdf (#91)

* chore: Update docker compose command

* fix(config): Force Rack attack to false

* Upgrade decidim-custom_proposal_states (#92)

* Update awesome with private fields (#93)

* update awesome with private fields

* use merged branch

* fix: migration FixReferenceForAllResources when table does not exists

* bump fix notifications and mails send after voting (#96)

* Bump: simple_proposal update (#97)

* Bump: simple_proposal update

* test: update system test

* Fix: Bump Decidim Awesome module (#95)

* Fix export dropdown in admin proposals index (#98)

* fix: add missing translation keys

* test: update config file after i18n test

* Bump simple proposal module (#100)

* fix: Add missing Sidekiq queues (#101)

* fix(sidekiq): duplicate sidekiq queue (#102)

The queue `user_report` is duplicated

* chore: Push version v0.4.1

---------

Co-authored-by: Guillaume MORET <[email protected]>
Co-authored-by: ’Barbara Oliveira <[email protected]>
Co-authored-by: Lucie Grau <[email protected]>
Co-authored-by: Alexandru Emil Lupu <[email protected]>
Co-authored-by: Ivan Vergés <[email protected]>
Co-authored-by: moustachu <[email protected]>
Co-authored-by: stephanierousset <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants